ci(frontend-deploy): name what is being deployed - #8431
talissoncosta wants to merge 2 commits into
Conversation
The cards identified a deploy by short sha, which says nothing about what is in it. Adds the pull request title under the headline, linked to the pull request. No API call is needed. A squash merge puts the title and number in the first line of the commit message, as "title (#123)", and all of the last forty commits on main match that. A direct push or a merge commit will not, so the number is optional and the title falls back to the whole line unlinked. The title goes full width as a context line rather than into the field grid, which is two narrow columns and would wrap a real title badly. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8431 +/- ##
========================================
Coverage 98.81% 98.82%
========================================
Files 1621 1622 +1
Lines 66118 66727 +609
========================================
+ Hits 65334 65942 +608
- Misses 784 785 +1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
📝 WalkthroughWalkthroughThe notification action now accepts repository and commit-message inputs. It derives a pull-request title and URL from the first commit-message line. The Slack payload conditionally displays the escaped title as a link or plain text. Both production deployment notification jobs provide the new inputs. Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: 🟡 Moderate · up to Long commit subjects can suppress deployment notifications, and commit-title text can create unintended Slack links. Bound the title and disable automatic parsing before merging. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Docker builds report
|
✅ private-cloud · depot-ubuntu-latest-16 — run #20400 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
🗂️ Previous results✅ private-cloud · depot-ubuntu-latest-arm-16 — run #20400 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
✅ oss · depot-ubuntu-latest-arm-16 — run #20400 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
✅ oss · depot-ubuntu-latest-16 — run #20400 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-16)Details
✅ private-cloud · depot-ubuntu-latest-arm-16 — run #20355 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
✅ private-cloud · depot-ubuntu-latest-16 — run #20355 (attempt 1)Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
✅ oss · depot-ubuntu-latest-arm-16 — run #20355 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
✅ oss · depot-ubuntu-latest-16 — run #20355 (attempt 1)Playwright Test Results (oss - depot-ubuntu-latest-16)Details
|
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 6ca7e52f-1f9f-4d93-9a26-c6f410cedb40
📒 Files selected for processing (3)
.github/actions/notify-slack-deploy/action.yml.github/actions/notify-slack-deploy/payload.jq.github/workflows/frontend-deploy-production.yml
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.
Visual Regression19 screenshots compared. See report for details. |
The title comes from the deployed commit subject, so it is the only free-form string in the card. A pull request titled "<!channel> ship it" would have pinged the whole channel on merge, and "<url|text>" would have rendered as a disguised link. Escapes the three characters Slack parses, ampersand first so the entities the other two produce are not re-encoded. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 4fc8ad10-98e4-40e9-9c81-dbd55da672bf
📒 Files selected for processing (1)
.github/actions/notify-slack-deploy/payload.jq
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
The deploy cards identify a deploy by short sha, which tells you nothing about what is in it. This adds the pull request title under the headline, linked to the pull request.
No API call and no extra permission. A squash merge puts the title and number in the first line of the commit message as
title (#123), and all forty of the last forty commits onmainmatch that shape. A direct push or a merge commit will not, so the number is optional: the title then falls back to the whole first line, unlinked, rather than producing a broken link.The title goes full width as a
contextblock rather than into the field grid, which is two narrow columns and would wrap a real title badly.Deploying
Deployed
How did you test this code?
The parsing, against five commit message shapes:
refactor(styles): diff view and code theme on shared tokens (#8069)/pull/8069chore: a direct push with no pull request/pull/8420fix: mentions (#123) mid-sentence and ends plainlyThat last case is why the pattern is anchored to the end of the line.
The rendering, across all four states with and without a title, confirming the
contextblock appears only when there is one and the block order is otherwise unchanged:Both files parse as YAML. Not exercised end to end, since this workflow only runs on push to
main. The steps arecontinue-on-error: true, so nothing here can hold up or fail a deploy.